Skip to content

Conversation

dsplatonov
Copy link
Contributor

Motivation:

AWS Lambda response streaming now supports a default maximum response payload size of 200 MB (increased from 20 MB). This update aligns the documentation with the new AWS limits.

Modifications:

  • Updated readme.md: Changed "soft limit of 20 MB" to "default maximum response payload size of 200 MB"
  • Updated Examples/Streaming/README.md: Same change as above
  • Updated terminology to match AWS official documentation

Result:

Documentation now accurately reflects the current AWS Lambda streaming response limits, enabling users to understand they can stream up to 200 MB payloads.

Reference: https://aws.amazon.com/about-aws/whats-new/2025/07/aws-lambda-response-streaming-200-mb-payloads/

- Updated readme.md and Examples/Streaming/README.md
- Changed from 'soft limit of 20 MB' to 'default maximum response payload size of 200 MB'
- Aligns with AWS announcement about increased Lambda streaming limits"
@sebsto
Copy link
Contributor

sebsto commented Aug 5, 2025

Thank you @dsplatonov !
Your change will fix #546

But I think there is a bit more than the doc to check. I know the LambdaRuntimeClient struct has several references to the 6Mb payload limit for non-streaming operations. We need to make sure there is no 20Mb limit hardcoded here and there.

@sebsto sebsto added the semver/none No version bump required. label Aug 5, 2025
@sebsto sebsto self-assigned this Aug 5, 2025
@sebsto sebsto added this to the 2.0 milestone Aug 5, 2025
@dsplatonov
Copy link
Contributor Author

Yea, I’ve checked all files, found 20 mb only in documentation, 6mb limit is not changed, so I didn’t change it in code:)

@sebsto sebsto self-requested a review August 6, 2025 07:57
Copy link
Contributor

@sebsto sebsto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I also checked for hard-code 20Mb limit in the code and could not find any.

@sebsto sebsto merged commit af7e9db into swift-server:main Aug 6, 2025
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants